Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add drawio support #158

Merged
merged 2 commits into from
Jun 20, 2022
Merged

Conversation

masahirominami
Copy link
Contributor

I wanted to experiment on the issue #21 , and here's my first try.
With this code .xml and .drawio files can be indexed.

If anyone is interested, any feedback will be welcome. Thank you.

@masahirominami masahirominami force-pushed the add-drawio-support branch 2 times, most recently from 691fd20 to d9040ea Compare April 20, 2022 03:34
@ArtificialOwl
Copy link
Member

Hello and thanks for your PR.

Can you share some files to test this ?

@masahirominami
Copy link
Contributor Author

Hi, thank you for the comment. Added test.xml and test.drawio along with the other existing "test.*" files, is this what you expected ? They should be found with keywords like "test-", "test-key", "test-value", "test-success". Draw.io format's sheet names are currently ignored.

@ArtificialOwl
Copy link
Member

Looks good, can you rebase on master ?

@masahirominami
Copy link
Contributor Author

Thanks for checking @ArtificialOwl , rebase done... but I must admit that I am still new to git operation. Hope things didn't get messed up..

@ArtificialOwl ArtificialOwl merged commit 0596a5c into nextcloud:master Jun 20, 2022
@ArtificialOwl
Copy link
Member

/backport to stable24

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants